Skip to content

Apply pipelines in OfficeController#33

Open
thefeqy wants to merge 1 commit intothemsaid:masterfrom
thefeqy:feqy-pipelines
Open

Apply pipelines in OfficeController#33
thefeqy wants to merge 1 commit intothemsaid:masterfrom
thefeqy:feqy-pipelines

Conversation

@thefeqy
Copy link

@thefeqy thefeqy commented Oct 17, 2021

Hi, I've added some pipelines that made the code much simple and also filtration classes that can be reusable.
ref: https://laravel.com/api/8.x/Illuminate/Pipeline/Pipeline.html
hope it can be useful.

all test passes!
Screenshot from 2021-10-17 15-52-45

finally I wanted to thank you for this amazing series.

best regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant